Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recommend Python Virtual Environment & Update Modules #255

Merged

Conversation

andrew-aiken
Copy link
Contributor

Overview of Changes

  • Recommend using python virtual environments
    • Git ignore the .venv directory
  • Updated python packages to latest versions

Testing

Locally running scenarios (including ecs_takeover)

Rational

Without setting up virtual environments the following error is raised.

❯ pip3 install -r ./requirements.txt
error: externally-managed-environment

× This environment is externally managed
╰─> To install Python packages system-wide, try brew install
    xyz, where xyz is the package you are trying to
    install.

    If you wish to install a non-brew-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip.

    If you wish to install a non-brew packaged Python application,
    it may be easiest to use pipx install xyz, which will manage a
    virtual environment for you. Make sure you have pipx installed.

Copy link
Contributor

@TeneBrae93 TeneBrae93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Verified the initial error and then set up a Python virtual environment and noted the error is fixed
  • Tested launching scenarios from the virtual environment with no issues

@TeneBrae93 TeneBrae93 merged commit f2a9dce into RhinoSecurityLabs:master Jun 12, 2024
@andrew-aiken andrew-aiken deleted the recommend-python-virtual-env branch June 12, 2024 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants